Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

Cleanup axelar config storage #203

Merged
merged 7 commits into from
Jul 18, 2023
Merged

Conversation

stevieraykatz
Copy link
Contributor

Ticket(s): #191

Explanation of the solution

We stored axelar gateway and gasRecv contract addresses in:

  • We stored the in the accounts config
  • Registrar stored network config
  • Router state (upon init)

We never used the accounts contract config storage versions so they've been easily removed.

The concept of Network was on the primary Registrar before but the Router is cross-chain. So for the Router to also use the Registrar as the source of truth, the storage and querying of networks needed to be moved down into the LocalRegistrar.

Instructions on making this work

  • run yarn or yarn install to install npm dependencies
  • run yarn test to verify all tests still pass

@stevieraykatz stevieraykatz self-assigned this Jul 17, 2023
@SovereignAndrey SovereignAndrey merged commit f9b5c65 into master Jul 18, 2023
1 check passed
@SovereignAndrey SovereignAndrey deleted the cleanup-axelar-config branch July 18, 2023 02:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants